Skip to content

fix(config): correct scoop_repo help URL#6649

Open
B67687 wants to merge 2 commits intoScoopInstaller:developfrom
B67687:fix/scoop-config-scoop-repo-help
Open

fix(config): correct scoop_repo help URL#6649
B67687 wants to merge 2 commits intoScoopInstaller:developfrom
B67687:fix/scoop-config-scoop-repo-help

Conversation

@B67687
Copy link
Copy Markdown

@B67687 B67687 commented Apr 29, 2026 •
edited
Loading

Summary

  • Update the documented scoop_repo default URL in scoop config --help to use HTTPS instead of HTTP
  • Correct typo: containining to containing
  • Make the help text match the actual default config value

Closes #6633.

Motivation and Context

The scoop config --help text showed an outdated HTTP URL for the default scoop_repo, which did not match the actual default value. This is a help-text-only fix with no runtime behavior changes.

How Has This Been Tested?

  • Ran scoop config --help and confirmed the output now shows the correct HTTPS URL
  • Confirmed the typo is corrected and the description is clearer

Checks

  • I have read the Contributing Guide.
  • I have ensured that I am targeting the develop branch.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly. (Not applicable in this case)
  • I have added an entry in the CHANGELOG. (Not applicable in this case)

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026 •
edited
Loading

Walkthrough

The pull request updates help/comment text in libexec/scoop-config.ps1: the scoop_repo example URL changes from http:// to https://, and a typo in a comment is corrected. No executable logic or public API was modified.

Changes

Cohort / File(s) Summary
Configuration Documentation
libexec/scoop-config.ps1		 Updated example scoop_repo URL from http://... to https://...; corrected comment typo ("containining" → "containing"). No behavioral changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

I'm a rabbit in the config glade,
I hopped from http to https today.
A tiny typo brushed away with care,
Help text now tidy, clean, and fair.
🐇🔒

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: correcting the scoop_repo help URL from HTTP to HTTPS to match the actual config default.
Linked Issues check ✅ Passed The PR fully addresses issue #6633 by updating the documented scoop_repo URL in help text from HTTP to HTTPS, matching the actual configured default value.
Out of Scope Changes check ✅ Passed All changes are directly related to resolving issue #6633: the URL update and typo fix are both within scope and necessary for the issue resolution.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai Bot reviewed Apr 29, 2026
View reviewed changes
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents 
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@libexec/scoop-config.ps1`:
- Around line 36-39: Fix the typo in the help text comment: change the word
"containining" to "containing" in the comment line "#       Git repository
containining scoop source code." so the help text reads "#       Git repository
containing scoop source code." This keeps the existing surrounding comments and
aligns the help text with the URL change already made.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0c5b1f99-ffbc-487e-9000-b51835d6e882

📥 Commits

Reviewing files that changed from the base of the PR and between a7a7d34 and b9b1309.

📒 Files selected for processing (1)
  • libexec/scoop-config.ps1

Comment thread libexec/scoop-config.ps1
@B67687
fix(config): correct scoop_repo help URL
eee819c 
- Change http:// to https:// for better security
- Fix typo in help text (containining → containing)
@B67687 B67687 force-pushed the fix/scoop-config-scoop-repo-help branch from 2739836 to eee819c Compare April 29, 2026 09:07
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@B67687 B67687 mentioned this pull request May 1, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@B67687
⚡